commit: Clarify that syncfs is of repo/tmp
authorColin Walters <walters@verbum.org>
Mon, 17 Mar 2025 23:09:53 +0000 (19:09 -0400)
committerColin Walters <walters@verbum.org>
Mon, 17 Mar 2025 23:10:42 +0000 (19:10 -0400)
We saw this in an error message:
```
error: Generating commit from rootfs: syncfs: Not a directory
```

I'm pretty sure it's this function call but let's be a bit
more sure by adding a bit more context.

src/libostree/ostree-repo-commit.c

index c5a47d32c833f5e1f3fccb6d0e5344bbd75a6de4..4bf845c40bd68ef75f6c596f0c00761b9ae4256d 100644 (file)
@@ -2233,7 +2233,7 @@ ostree_repo_commit_transaction (OstreeRepo *self, OstreeRepoTransactionStats *ou
   if (!self->disable_fsync && g_getenv ("OSTREE_SUPPRESS_SYNCFS") == NULL)
     {
       if (syncfs (self->tmp_dir_fd) < 0)
-        return glnx_throw_errno_prefix (error, "syncfs");
+        return glnx_throw_errno_prefix (error, "syncfs(repo/tmp)");
     }
 
   if (!rename_pending_loose_objects (self, cancellable, error))